From 464078619ab5c8db2d87398dfe50897f94c3eea9 Mon Sep 17 00:00:00 2001 From: "Richard M. Stallman" Date: Sun, 6 Jun 1993 22:47:15 +0000 Subject: [PATCH] (same_size_fonts): Compare only the width. --- src/xfaces.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/xfaces.c b/src/xfaces.c index fc56bb4a8a7..7a93111f57d 100644 --- a/src/xfaces.c +++ b/src/xfaces.c @@ -590,9 +590,16 @@ same_size_fonts (font1, font2) XCharStruct *bounds1 = &font1->min_bounds; XCharStruct *bounds2 = &font2->min_bounds; - return (bounds1->width == bounds2->width + return (bounds1->width == bounds2->width); +/* Checking the following caused bad results in some cases + when fonts that should be the same size + actually have very slightly different size. + It is possible that this reintroduces the bug whereby line positions + were not right. However, the right way to fix that is to change xterm.c + so that the vertical positions of lines + depend only on the height of the frame's font. && bounds1->ascent == bounds2->ascent - && bounds1->descent == bounds2->descent); + && bounds1->descent == bounds2->descent); */ } /* Modify face TO by copying from FROM all properties which have -- 2.30.2